-
Notifications
You must be signed in to change notification settings - Fork 581
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ERC: Quantum Supremacy Bounty #731
base: master
Are you sure you want to change the base?
Conversation
ERCS/erc-X.md
Outdated
@@ -0,0 +1,550 @@ | |||
--- | |||
eip: X | |||
title: Quantum Supremacy Puzzle |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps Quantum supremacy canary or bounty might be better than puzzle
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated to "Quantum Supremacy Bounty".
ERCS/erc-X.md
Outdated
@@ -0,0 +1,550 @@ | |||
--- | |||
eip: X |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eip: X | |
eip: 7826 |
Assigning next sequential EIP/ERC/RIP number.
Numbers are assigned by editors & associates.
Please also update the filename.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated!
ERCS/erc-X.md
Outdated
eip: X | ||
title: Quantum Supremacy Puzzle | ||
author: Nicholas Papadopoulos (@nikojpapa) | ||
discussions-to: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please create a discussions topic on Eth Magicians with a link to this PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated with the link (https://ethereum-magicians.org/t/erc-7826-quantum-supremacy-bounty/21866).
Is there a prior example of adding an entire sol repo with dependencies to the assets folder? |
This looks like the largest set of assets compared to other ERCs, though it does not seem uncommon to include implementations (e.g., 6358, 6220, 5252, etc.). We may not need all of the assets in this ERC, but they do provide proof of the statements contained in this ERC, including tests and comparisons to other potential puzzles. |
The commit 29d3f91 (as a parent of e903909) contains errors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This proposal doesn't require an EIP/ERC. Simply deploy the contract.
@@ -0,0 +1,74 @@ | |||
// SPDX-License-Identifier: GPL-3.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We cannot accept GPL'd code. It would carry over to any implementation based on this reference implementation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Which license would you recommend? Or should we just remove this line altogether?
@@ -0,0 +1,21 @@ | |||
// SPDX-License-Identifier: GPL-3.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
@@ -0,0 +1,123 @@ | |||
// SPDX-License-Identifier: GPL-3.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
@@ -0,0 +1,20 @@ | |||
// SPDX-License-Identifier: GPL-3.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
@@ -0,0 +1,45 @@ | |||
// SPDX-License-Identifier: GPL-3.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Big reference implementation! For all the files, no GPL.
Quantum supremacy[^1] is a demonstration of a quantum computer solving a problem that would take a classical computer an infeasible amount of time to solve. | ||
Previous attempts have been made to demonstrate quantum supremacy, e.g. Kim[^2], Arute[^3] and Morvan[^4], | ||
but they have been refuted or at least claimed to have no practical benefit, e.g. Begusic and Chan[^5], Pednault[^6], | ||
and a quote from Sebastian Weidt (The Telegraph, "Supercomputer makes calculations in blink of an eye that take rivals 47 years", 2023). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please cite external documents using the formats allowed in EIP-1. If you cannot, please either propose a new origin using the process from EIP-5757 or remove this link.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cannot cite it since there is no DOI, hence I just gave the name of the article as plain text if anyone wanted to search for it. I thought this would prevent the issues of defunct links and such. I do think there should be some sort of reference so people can confirm the quote.
When discussing this with Danny Ryan (@djrtwo), we decided that this should be deployed as a singleton as it should be a single instance across any chain. So, wouldn't this require ERC-2470: Singleton Factory? Although, I see that it is in a "stagnant" stage, so I can see why we might not want to mark it as dependent on that. Would you still like me to remove the explicit dependency but still require deploying as a singleton? |
No description provided.